-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[WIP] WASM #15
base: master
Are you sure you want to change the base?
[WIP] WASM #15
Conversation
* readme lints * update readme * fix remove sdl timer * fixed codacy issues * Update README.md
* refacto cmake tests file * uinittest update * .. * Rename README.md to Readme.md
* refactor cmake object name to vga-terminal-obj plust add runtime test for SO * move examples into example dir * [cmake] add static lib and refactor install rules * static lib to be linked with static sdl2 as option off * fixed cmake exports for static lib * rename readme * Apply suggestions from code review
* autoscroll flag * vgaterminal getMode method * code review
Here is an overview of what got changed by this pull request: Issues
======
+ Solved 19
- Added 2
Clones added
============
- sdl2-vga-terminal/examples/wasm.cpp 1
See the complete overview on Codacy |
printf("loading library: %s\n", soname); | ||
|
||
hinstLib = LoadLibrary(TEXT("vga-terminal-libd.dll")); | ||
hinstLib = LoadLibrary(soname); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
uint8_t* font; | ||
int numColors; | ||
uint8_t* palette; // BGR palette assumed (might be required a palette format flag?) | ||
uint8_t* palette; // RGB palette assumed (might be required a palette format flag?) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Issue found: struct member 'videoMode_t::palette' is never used.
No description provided.